Skip to content

feat: add Rejected Proposals page (#166)#167

Merged
rdmueller merged 2 commits into
LLM-Coding:mainfrom
raifdmueller:feat/rejected-proposals-page
Mar 9, 2026
Merged

feat: add Rejected Proposals page (#166)#167
rdmueller merged 2 commits into
LLM-Coding:mainfrom
raifdmueller:feat/rejected-proposals-page

Conversation

@raifdmueller

@raifdmueller raifdmueller commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • New "Rejected Proposals" page listing evaluated but not qualified anchor proposals
  • Three rejection categories: "Structurally unsuitable", "Not in training data", "Too niche"
  • Each category links to the quality criteria on the About page
  • Initial entries: TLDR, ELI5, MIRRR UX Framework ([Anchor Proposal]: MIRRR UX Framework #150)
  • Bilingual (EN + DE)
  • Linked from footer and Contributing page (not in main nav)
  • "How We Evaluate" section explains the testing process

Closes #166

Test plan

  • npm test — 87 tests passing
  • render-docs.js generates rejected-proposals.html and .de.html
  • Footer link present
  • Contributing page references rejected proposals
  • Language switch works

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Neue Features

    • Neue Seite "Abgelehnte Vorschläge" verfügbar (DE/EN) mit Kategorien und Ablehnungsgründen; per Footer-Link erreichbar.
  • Dokumentation

    • Beitragsleitfaden um Hinweise zur Seite ergänzt; bilingual verfügbare Dokumentation und Beispiele zur Bewertungsmethodik hinzugefügt.
  • Chores

    • Vorab-Rendering für die neue Seite integriert, Übersetzungstexte ergänzt und Git‑Ignorierregeln aktualisiert.

Closes LLM-Coding#166

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: 6cc4b8f1-c44c-475e-b194-89da928309af

📥 Commits

Reviewing files that changed from the base of the PR and between efd5121 and 571dd62.

📒 Files selected for processing (1)
  • .gitignore

Walkthrough

Eine neue, bilingual gepflegte Seite "Rejected Proposals" wurde hinzugefügt (AsciiDoc-Quellen, deutsche Übersetzung), in die Rendering-Pipeline aufgenommen, ins Website-Routing und Footer integriert sowie in die Contributing-Dokumentation referenziert.

Changes

Cohort / File(s) Summary
Dokumentation
docs/rejected-proposals.adoc, docs/rejected-proposals.de.adoc, docs/plans/2026-03-09-rejected-proposals-design.md
Neue Seiten/Design-Dokument: Kategorisierung abgelehnter Proposals, Beispiel-Tabelle, Bewertungs-Kriterien; Design/Scope und initiale Einträge.
Contributing Guide
CONTRIBUTING.adoc
Zwei TIP-Notizen hinzugefügt, die auf die Rejected Proposals-Seite verweisen (keine Logikänderung).
Rendering & Build
scripts/render-docs.js
Zwei neue renderFile-Aufrufe für rejected-proposals.adoc und rejected-proposals.de.adoc hinzugefügt (Vor-Render-Integration).
Website-Integration
website/src/main.js, website/src/components/footer.js
Neue Route /rejected-proposals mit renderRejectedProposalsPage() registriert; Footer um Link ergänzt; Seitenlade-/Sprachwechsel-Handling erweitert.
Lokalisierung
website/src/translations/en.json, website/src/translations/de.json
Neue Übersetzungs-Keys footer.rejectedProposals (EN/DE) hinzugefügt.
Sonstiges VCS
.gitignore
Ignoriere *:Zone.Identifier und vorgerenderte website/public/CONTRIBUTING*.html.

Sequence Diagram

sequenceDiagram
    actor User
    participant Footer as "Footer Navigation"
    participant Router as "Router (website/src/main.js)"
    participant Renderer as "renderRejectedProposalsPage()"
    participant Docs as "docs/rejected-proposals(.de).adoc"
    participant i18n as "i18n / translations"

    User->>Footer: Klick "Rejected Proposals"
    Footer->>Router: Navigiere zu /rejected-proposals
    Router->>Renderer: Aufruf renderRejectedProposalsPage()
    Renderer->>Docs: Lade gerenderte HTML/AsciiDoc-Inhalt
    Renderer->>i18n: Frage lokalisierte Strings an
    i18n->>Renderer: Liefere Übersetzungen
    Renderer->>User: Rendere Seite
    User->>i18n: Sprachenwechsel
    i18n->>Docs: Lade docs/rejected-proposals.de.adoc
    Renderer->>User: Aktualisierte lokalisierte Seite anzeigen
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Die Überschrift 'feat: add Rejected Proposals page (#166)' beschreibt präzise die Hauptänderung: eine neue Seite für abgelehnte Vorschläge wird hinzugefügt. Sie ist klar, spezifisch und relevant für die gesamte Änderung.
Linked Issues check ✅ Passed Der PR erfüllt alle Anforderungen aus Issue #166: eine bilingual (EN/DE) Seite mit abgelehnten Vorschlägen, kategorisiert nach Ablehnungsgründen, mit Links in Footer und Contributing-Seite, und drei initialen Beispiele (TLDR, ELI5, MIRRR #150).
Out of Scope Changes check ✅ Passed Alle Änderungen sind direkt mit dem Ziel verbunden: neue Dokumentseite, Routing, Übersetzungen, Footer-Link und Build-Konfiguration für die Rejected Proposals-Seite. Keine irrelevanten Änderungen erkannt.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@rdmueller rdmueller merged commit 2125995 into LLM-Coding:main Mar 9, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: 'Rejected Proposals' page for evaluated but not qualified anchors

2 participants